Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

EVEREST-633 Install monitoring stack onto a separate namespace #281

Merged

Conversation

recharte
Copy link
Collaborator

@recharte recharte commented Feb 6, 2024

Install monitoring stack onto a separate namespace

Problem:
EVEREST-633

By moving to a multi-namespace everest-operator installation we configured the operator group of the percona-everest namespace to have N target namespaces.
The VM Operator doesn't support multi-namespaces so we can't deploy it to the same namespace as the everest-operator.
Therefore we install the monitoring stack onto a separate namespace for managing everything monitoring related.

CHECKLIST

Jira

  • Is the Jira ticket created and referenced properly?

Tests

  • [ ] Is an Integration test/test case added for the new feature/change?
  • [ ] Are unit tests added where appropriate?

@recharte recharte changed the base branch from main to EVEREST-495-multi-namespaces February 6, 2024 09:35
@recharte recharte force-pushed the EVEREST-633-multi-namespaces-monitoring branch from 8a9ac71 to df75192 Compare February 6, 2024 09:37
pkg/install/install.go Outdated Show resolved Hide resolved
pkg/install/install.go Outdated Show resolved Hide resolved
By moving to a multi-namespace everest-operator installation we
configured the operator group of the percona-everest namespace to have N
target namespaces. The VM Operator doesn't support multi-namespaces so
we can't deploy it to the same namespace as the everest-operator.
Therefore we install it onto a separate namespace for managing
everything monitoring related.
@recharte recharte force-pushed the EVEREST-633-multi-namespaces-monitoring branch from 020692d to dd24375 Compare February 6, 2024 09:48
@codecov-commenter
Copy link

codecov-commenter commented Feb 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (EVEREST-495-multi-namespaces@c9915ac). Click here to learn what that means.

Additional details and impacted files
@@                      Coverage Diff                       @@
##             EVEREST-495-multi-namespaces    #281   +/-   ##
==============================================================
  Coverage                                ?   5.31%           
==============================================================
  Files                                   ?      12           
  Lines                                   ?    2693           
  Branches                                ?       0           
==============================================================
  Hits                                    ?     143           
  Misses                                  ?    2525           
  Partials                                ?      25           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@recharte recharte marked this pull request as ready for review February 6, 2024 10:07
@recharte recharte requested a review from a user February 6, 2024 10:07
pkg/install/install.go Outdated Show resolved Hide resolved
pkg/install/install.go Show resolved Hide resolved
pkg/install/install.go Show resolved Hide resolved
@recharte recharte force-pushed the EVEREST-633-multi-namespaces-monitoring branch from 1972bc6 to 1675d32 Compare February 7, 2024 10:55
@recharte recharte merged commit d89b7df into EVEREST-495-multi-namespaces Feb 7, 2024
10 checks passed
@recharte recharte deleted the EVEREST-633-multi-namespaces-monitoring branch February 7, 2024 16:46
recharte added a commit that referenced this pull request Feb 7, 2024
* EVEREST-633 install monitoring stack with install command

By moving to a multi-namespace everest-operator installation we
configured the operator group of the percona-everest namespace to have N
target namespaces. The VM Operator doesn't support multi-namespaces so
we can't deploy it to the same namespace as the everest-operator.
Therefore we install it onto a separate namespace for managing
everything monitoring related.

* EVEREST-633 remove monitoring command

* EVEREST-633 Update everest-operator go mod

* EVEREST-633 Update percona-everest-backend go mod

* EVEREST-633 forbid installs in the monitoring namespace

* EVEREST-633 update namespaces env var name

* EVEREST-633 Update everest-operator go mod
@recharte recharte mentioned this pull request Feb 7, 2024
3 tasks
recharte added a commit that referenced this pull request Feb 7, 2024
* EVEREST-633 Add multi-namespace support (#271)

* EVEREST-633 force everest install to percona-everest namespace

* EVEREST-633 remove deprecated name flag

* EVEREST-633 add multi-namespace support

* EVEREST-633 remove operator channel flags

* EVEREST-633 update backend and operator go mods

* EVEREST-633 shut up linter, we know this code is dead

* EVEREST-633 Install monitoring stack onto a separate namespace (#281)

* EVEREST-633 install monitoring stack with install command

By moving to a multi-namespace everest-operator installation we
configured the operator group of the percona-everest namespace to have N
target namespaces. The VM Operator doesn't support multi-namespaces so
we can't deploy it to the same namespace as the everest-operator.
Therefore we install it onto a separate namespace for managing
everything monitoring related.

* EVEREST-633 remove monitoring command

* EVEREST-633 Update everest-operator go mod

* EVEREST-633 Update percona-everest-backend go mod

* EVEREST-633 forbid installs in the monitoring namespace

* EVEREST-633 update namespaces env var name

* EVEREST-633 Update everest-operator go mod

* EVEREST-633 Use PGO stable channel (#282)

* EVEREST-633 install monitoring stack with install command

By moving to a multi-namespace everest-operator installation we
configured the operator group of the percona-everest namespace to have N
target namespaces. The VM Operator doesn't support multi-namespaces so
we can't deploy it to the same namespace as the everest-operator.
Therefore we install it onto a separate namespace for managing
everything monitoring related.

* EVEREST-633 remove monitoring command

* EVEREST-633 Update everest-operator go mod

* EVEREST-633 Update percona-everest-backend go mod

* EVEREST-633 forbid installs in the monitoring namespace

* EVEREST-633 use PGO stable channel

* EVEREST-495 Update everest-operator go mod

* EVEREST-495 Fix missing monitoring namespace env var
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants